Skip to content

RHIDP-9739: Orchestrator Permissions #1682

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
deerskindoll merged 4 commits into from
Jan 19, 2026
Merged

RHIDP-9739: Orchestrator Permissions #1682

deerskindoll merged 4 commits into from
Jan 19, 2026
+152 −1

Conversation

@jmagak
Copy link
Member

@jmagak jmagak commented Jan 16, 2026 •
edited
Loading

IMPORTANT: Do Not Merge - To be merged by Docs Team Only

Version(s): main, 1.8, 1.7

Issue: RHIDP-9739

Preview:

3.4. Orchestrator plugin permissions
3.5. Managing Orchestrator plugin permissions using RBAC policies

@jmagak jmagak force-pushed the RHIDP-9739-Orchestrator-Permissions branch from 5eaa6ae to 5c7fab6 Compare January 16, 2026 14:48
@rhdh-bot
Copy link
Collaborator

rhdh-bot commented Jan 16, 2026 •
edited
Loading

Updated preview: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-1682/ @ 01/19/26 14:43:24

@jmagak jmagak requested a review from chadcrum January 16, 2026 15:12
@jmagak jmagak added Technical review done ⛅ Any procedure has been succesfully tested and removed Technical review needed 🔩 Test all the procedures labels Jan 19, 2026
pabel-rh
pabel-rh reviewed Jan 19, 2026
View reviewed changes
Copy link
Member

@pabel-rh pabel-rh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some comments

pabel-rh
pabel-rh reviewed Jan 19, 2026
View reviewed changes
pabel-rh
pabel-rh approved these changes Jan 19, 2026
View reviewed changes
Copy link
Member

@pabel-rh pabel-rh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with minor changes

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 19, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@jmagak
Copy link
Member Author

jmagak commented Jan 19, 2026

/cherry-pick release-1.7

@openshift-cherrypick-robot
Copy link
Contributor

openshift-cherrypick-robot commented Jan 19, 2026

@jmagak: once the present PR merges, I will cherry-pick it on top of release-1.7 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-1.7

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@jmagak
Copy link
Member Author

jmagak commented Jan 19, 2026

/cherry-pick release-1.8

@openshift-cherrypick-robot
Copy link
Contributor

openshift-cherrypick-robot commented Jan 19, 2026

@jmagak: once the present PR merges, I will cherry-pick it on top of release-1.8 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-1.8

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@deerskindoll deerskindoll merged commit bfcb405 into redhat-developer:main Jan 19, 2026
6 checks passed
@openshift-cherrypick-robot
Copy link
Contributor

openshift-cherrypick-robot commented Jan 19, 2026

@jmagak: #1682 failed to apply on top of branch "release-1.8":

Applying: Update Orchestrator plugin permissions
Using index info to reconstruct a base tree...
M	assemblies/assembly-install-rhdh-orchestrator-operator.adoc
Falling back to patching base and 3-way merge...
Auto-merging assemblies/assembly-install-rhdh-orchestrator-operator.adoc
CONFLICT (content): Merge conflict in assemblies/assembly-install-rhdh-orchestrator-operator.adoc
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Update Orchestrator plugin permissions
Details

In response to this:

/cherry-pick release-1.8

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-cherrypick-robot
Copy link
Contributor

openshift-cherrypick-robot commented Jan 19, 2026

@jmagak: #1682 failed to apply on top of branch "release-1.7":

Applying: Update Orchestrator plugin permissions
Using index info to reconstruct a base tree...
M	assemblies/assembly-install-rhdh-orchestrator-operator.adoc
Falling back to patching base and 3-way merge...
Auto-merging assemblies/assembly-install-rhdh-orchestrator-operator.adoc
CONFLICT (content): Merge conflict in assemblies/assembly-install-rhdh-orchestrator-operator.adoc
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Update Orchestrator plugin permissions
Details

In response to this:

/cherry-pick release-1.7

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pabel-rh pabel-rh pabel-rh approved these changes

@chadcrum chadcrum Awaiting requested review from chadcrum

+1 more reviewer

@mareklibra mareklibra mareklibra approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

do-no-merge/review-in-progress 👀 Peer review done 📘 Technical review done ⛅ Any procedure has been succesfully tested

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@jmagak @rhdh-bot @openshift-cherrypick-robot @mareklibra @pabel-rh @deerskindoll